-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
API cleanup #58
API cleanup #58
Conversation
1487175
to
9ed4803
Compare
@@ -18,7 +17,7 @@ spec: | |||
- name: v1beta1 | |||
schema: | |||
openAPIV3Schema: | |||
description: Package | |||
description: Package is the Schema for the packages API |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we use apostrophe for possessives in the spec?
description: Package is the Schema for the packages API | |
description: Package is the Schema for the package's API |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is meant to be plural (e.g. the resource name).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
In preparation for more drastic changes to our APIs, I've gone through and cleaned up stuff that is not currently used. This should make it easier to review future substantive changes to the APIs.
This PR also changes the
BundleMetadata
property type to useruntime.RawExtension
instead of[]byte
, which is necessary to correctly serialize/deserizalize property values. (discussed here)